Skip to content

feat(gamecounts): add mode filter option#842

Open
ugcodrr wants to merge 2 commits into
mainfrom
feat/gamecounts-mode-filter
Open

feat(gamecounts): add mode filter option#842
ugcodrr wants to merge 2 commits into
mainfrom
feat/gamecounts-mode-filter

Conversation

@ugcodrr
Copy link
Copy Markdown
Member

@ugcodrr ugcodrr commented May 12, 2026

Summary

  • add an optional autocompleted mode option to /gamecounts
  • render the selected game's counts directly when mode is provided
  • keep the existing overall paginated gamecounts view when mode is omitted
  • fix command change detection so adding options to commands that previously had none triggers command reposting

@vercel
Copy link
Copy Markdown

vercel Bot commented May 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
statsify Skipped Skipped May 30, 2026 9:46am

* Change selectedGame type to string for better flexibility
* Update game count pagination to handle selectedGame validation
* Enhance createGameCountsPage to optionally show title emoji
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a /gamecounts autocomplete option for selecting a specific game and fixes command change detection so commands repost when options are added to previously optionless commands.

Changes:

  • Adds a mode autocomplete argument to /gamecounts.
  • Renders a single selected game count page when a valid mode is provided, while preserving the existing overall paginated view.
  • Updates command equality logic and adds test coverage for detecting newly added arguments.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
apps/discord-bot/src/commands/gamecounts.command.ts Adds gamecount mode autocomplete and selected-game rendering.
packages/discord/src/command/command.resolvable.ts Fixes option count comparison during command equality checks.
packages/discord/src/command/command.builder.ts Adds a test for detecting added command arguments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants